Skip to content

Workaround for Cython issue on Python 3.8#2684

Merged
postrational merged 3 commits intoopenvinotoolkit:masterfrom
postrational:mk/cython_fix
Oct 16, 2020
Merged

Workaround for Cython issue on Python 3.8#2684
postrational merged 3 commits intoopenvinotoolkit:masterfrom
postrational:mk/cython_fix

Conversation

@postrational
Copy link
Contributor

This is a proposed workaround for an issue we're seeing with Cython when using Python 3.8 to build the IE Python API.

openvino/build-test/inference-engine/ie_bridges/python/src/openvino/inference_engine/constants.cxx:3409:3: error: 'tp_print' is deprecated [-Werror,-Wdeprecated-declarations]
  0, /*tp_print*/
  ^
/usr/local/Frameworks/Python.framework/Versions/3.8/include/python3.8/cpython/object.h:260:5: note: 'tp_print' has been explicitly marked deprecated here
    Py_DEPRECATED(3.8) int (*tp_print)(PyObject *, FILE *, int);
    ^
/usr/local/Frameworks/Python.framework/Versions/3.8/include/python3.8/pyport.h:515:54: note: expanded from macro 'Py_DEPRECATED'
#define Py_DEPRECATED(VERSION_UNUSED) __attribute__((__deprecated__))
...
...
2 errors generated.

Related to: cython/cython#3474

@postrational postrational requested review from a team, ilya-lavrenov and tsocha October 15, 2020 17:03
@postrational postrational added the category: Python API OpenVINO Python bindings label Oct 15, 2020
@postrational postrational merged commit ddd9cf6 into openvinotoolkit:master Oct 16, 2020
@postrational postrational deleted the mk/cython_fix branch October 16, 2020 06:44
mryzhov pushed a commit to mryzhov/openvino that referenced this pull request Dec 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: Python API OpenVINO Python bindings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants